Add more label parsing tests

Test various combinations of underline and markup
parsing.

label: Ignore __ for mnemonics

We were not ignoring double underscores for mnemonics
in one place, as we should.

Fixes: #4129

Merge branch 'titlebar-gesture' into 'master'

Delegate titlebar action to the compositor if possible

See merge request GNOME/gtk!3797

Update Polish translation

Merge branch 'color-glyph-fixes' into 'master'

gsk: Make color glyphs

Closes #4141

See merge request GNOME/gtk!3812

Merge branch 'fix-iconhelper' into 'master'

iconhelper: Fix the paintable implementation

Closes #4140

See merge request GNOME/gtk!3809

gsk: Use harfbuzz for color fonts

harfbuzz has all the information we need, so we
can avoid poking directly at freetype apis. Also
drop the caching of color glyph information until
it turns out to be a problem.

Bump the harfbuzz dep

We need 2.1.0, which has the hb_ot_color apis that we
are going to use in the next commit.

gsk: Cache per-glyph color information

We don't really want to load the bitmaps every time
we create a render node, so do it once and cache the
information on the PangoFont.

ngl: Use per-glyph color information

Decide per-glyph whether we need color nor not.

Fixes: #4141

gsk: Mark color glyphs

Steal a bit from PangoGlyphVisAttr to mark glyphs
that have color.

Merge branch 'update-docs' into 'master'

docs: Update the URL for related documentation

See merge request GNOME/gtk!3813

docs: Update the URLs of related projects for GTK

And install the gi-docgen project file as well.

docs: Update the URLs of related projects for GSK

And install the gi-docgen project file.

docs: Update the URL for related documentation

And install the gi-docgen project files for GDK.

Merge branch 'docs-trigger' into 'master'

ci: Trigger a pages build

See merge request GNOME/gtk!3811

ci: Trigger a pages build

Do not publish the API reference directly, but trigger a CI pipeline for
the docs-gtk-org branch, which will then do the publishing.

Merge branch 'iter-is-valid-docs' into 'master'

docs: Fix iter_is_valid() description

See merge request GNOME/gtk!3810

docs: Fix iter_is_valid() description

Drop the "WARNING" and the block quote, and follow the existing
documentation style guidelines.

iconhelper: Fix the paintable implementation

get_current_image was recursing due to an obvious typo.

Fixes: #4140

Merge branch 'glyph-cache-mystery' into 'master'

ngl: Bypass pango for glyph cache rendering

See merge request GNOME/gtk!3807

ngl: Bypass pango for glyph cache rendering

We can just use cairo directly here,
and cut out some layers of pango in the middle.

Merge branch 'check-activate' into 'master'

checkbutton: Some fixes when used with GActions

See merge request GNOME/gtk!3805

Merge branch 'fix/wm_class' into 'master'

x11: set a default value for program_class

Closes #4138

See merge request GNOME/gtk!3806

Merge branch 'wip/otte/for-master' into 'master'

widget-factory: Expand the video to cover the whole frame

See merge request GNOME/gtk!3804

x11: set a default value for program_class

This was also the behaviour with GTK 3. We use the capitalized program
name.

Fixes #4138

Signed-off-by: Vincent Bernat <vincent@bernat.ch>

widget-factory: Propagate natural size for text styles

This avoids them requiring scrolling.

widget-factory: Make frames cooler

Instead of havoing a label for the video frame that clashes with the
background of the video, add a frame around the text styles box and add
a label for them. As a side benefit, it also makes it more obvious that
it is scrollable.

Note: Most of this patch is just reindenting.

widget-factory: Expand the video to cover the whole frame

1. This looks way cooler.

2. Since the frame is rounded, it adds a very visible test for our
   clipping of rounded rectangles.

checkbutton: Activate when moving focus

Currently we update the :active property on both the previous and
new focus button. That "visually activate" the button and will
emit ::toggled, but if the button is associated with an action,
the action state won't change.

Fix that by activating the new focus instead of explicitly fiddling
with the :active property.

checkbutton: Delegate ::activate to the helper if appropriate

If the button is associated with an action, the action is currently
only activated when the button is clicked, not when it is activated.

Merge branch 'wip/chergert/fix-4134' into 'master'

textview: fix yoffset position when top_margin is set

Closes #4134

See merge request GNOME/gtk!3801

textview: fix yoffset position when top_margin is set

This doesn't need to be included in the calculation or it will cause the
yoffset to continually shift while the window is resized.

Fixes #4134

gtk/windowhandle: Delegate titlebar action to the compositor if possible

Delegating the action to the compositor not only improves consistency
with server-side decorations, but also allows for actions that aren't
available client-side (like lower-in-middle-click).

https://gitlab.gnome.org/GNOME/mutter/-/issues/602

gdk/toplevel: Add gdk_toplevel_titlebar_gesture()

Where supported, this allows delegating the titlebar action to the
compositor instead of trying to replicate its behavior client-side.

https://gitlab.gnome.org/GNOME/mutter/-/issues/602

Merge branch 'source-static-name' into 'master'

Avoid copying static debug strings

See merge request GNOME/gtk!3789

ci: Collect tools test output

This should help understanding why some of them fail.

testsuite: Set TEST_RESULT_DIR for tools tests

Avoid copying static debug strings

The g_source_set_name calls were showing up as a
major source of strdups in our profiles. Avoid that
by using new GLib api when available.

Merge branch 'compose-parse-msvc' into 'master'

compose-parse.c: Include stdlib.h (fix build on Visual Studio)

See merge request GNOME/gtk!3794

Update Friulian translation

(cherry picked from commit 16aa8dbad74ac7dd29fa2e807e57c34f00eef3a7)

compose-parse.c: Include stdlib.h

That is needed for exit(), and it may not have been included by the
other headers.

Merge branch 'gi-fix' into 'master'

gdk: add missing Since annotation

See merge request GNOME/gtk!3790

gdk: add missing Since annotation

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'wip/otte/widgetfactory-dnd' into 'master'

Improve DND

See merge request GNOME/gtk!3785

widget-factory: Add images that support dnd

Just use the bottom left notebook for a GtkPicture with custom drag
source and drop target.

widget-factory: Some new backgrounds

Snowy wasn't ideal because of the aspect ratio.
Replace it by beach and nyc.

droptarget: Make formats construct-only writable

This way, it can be set in GtkBuilder.

Also make sure to only ever look at the GTypes set in the formats, as
GtkDropTarget cannot deal with mime types.

builder: Add support for GdkContentFormats

testsuite: Add contentformat tests

contentformats: Add gdk_content_formats_parse()

Merge branch 'matthiasc/for-master' into 'master'

ngl: Cosmetics

See merge request GNOME/gtk!3786

ngl: Cosmetics

Use the macro we already have to check that colors
are fully transparent.

Merge branch 'wip/compose-parser' into 'master'

Clean up the GtkComposeTable api

See merge request GNOME/gtk!3746

contentformats: Change the format of gdk_content_formats_to_string()

Now, we just print a whitespace-separated list of GTypes and mime types.

This makes this neat for 2 things:

1. Parsing it (see next commit)
2. Using it in GtkBuilder (see commits after that)

In particular, the common case of supporting a single GType (or mime
type) looks like just printing the GType (or mime type), which in
GtkBuilder looks like
<property name="formats">GdkTexture</property>

dnd: Guard drags with a reference when finishing

Usually the "dnd-finished" signal will be used to unref the GdkDrag. In
those cases, we would lose the object, so that when we do the final
drag_drop_done() afterwards, we wouldn't have a remaining reference.

With the reference guard, this now works.

4.3.2

Merge branch 'misc-speedups' into 'master'

ngl: Speed up clipping calculations a bit

See merge request GNOME/gtk!3784

sizerequest: Simplify some code

Avoid some repeated MAX().

sizerequest: Keep branches in sync

There are some unintentional differences between
the path that uses a layout manager, and the path
that uses widget vfuncs. Sync them up.

widget: Avoid some unnecessary back-and-forth casting

Avoid some unnecessary casting and a redundant early exit.

widget: Avoid GObject data for action muxer

Most widget have an action muxer now, so just
store it in GtkWidgetPrivate.

layoutmanager: Avoid some overhead

Use inline getters for some things.

gridlayout: Avoid some unnecessary overhead

Use inline functions for some things.

gsk: Optimize gsk_transform_to...

Optimize these functions for the common cases.

gsk: Add a shortcut for a common transform case

Add a private getter for the translation part of a
transform node, to avoid pulling it out of the transform
over and over.

gsk: Add a private getter for container children

No need to get them one-by-one, in our inner loop.

ngl: Reduce type checking overhead

No need to be very careful here, these are entirely
internal types.

ngl: Speed up clipping calculations a bit

Exit as early as possible from intersect_rounded_rectilinear.

Update POTFILES.in

Merge branch 'no-pposition' into 'master'

gdk/x11: Do not set PPosition hint

See merge request GNOME/gtk!3781

gdk/x11: Do not set PPosition hint

It is good practice for (floating) window managers to respect explicit
position hints from clients (as long as the window wouldn't end up
off-screen etc.).

Before commit 13d3afa56e9, GTK had a flag for setting the PPosition hint,
but now does so unconditionally. However the real intention is to *not*
request a fixed position, so don't do that.

Merge branch 'wip/exalm/gtktext-selection-fix' into 'master'

text: Use the widget y coordinate for gestures

See merge request GNOME/gtk!3780

text: Use the widget y coordinate for gestures

Currently we use layout coordinates and widget height when determining
where a click or drag has happened. If the widget has top padding (which it
does inside a GtkEntry, for example), the area where it's possible to select
text is shifted down, so the part of GtkText above the layout is not counted
as the draggable area and instead the equal area below the widget is counted.

Since GtkText is always single-line, there's no need to do any of that and
we can use widget coordinates. Then the draggable area matches the widget
and the problems goes away.

Merge branch 'stack-page-name' into 'master'

stack: GtkStackPage:name is not construct-only

Closes #4109

See merge request GNOME/gtk!3778

Merge branch 'matthiasc/for-master' into 'master'

Cosmetics

See merge request GNOME/gtk!3779

Cosmetics

Fix a comment typo.

stack: GtkStackPage:name is not construct-only

We have a setter for this property, so we clearly
weren't serious when we marked it as construct-only.

Fixes: #4109

Merge branch 'startup-profiling' into 'master'

Add more sysprof marks

See merge request GNOME/gtk!3777

Add more sysprof marks

Add a few more marks during gtk_init to figure out where
our startup time goes, and avoid the sysprof initialization
from distorting the first mark.

Merge branch 'startup-profiling' into 'master'

Add more sysprof marks

See merge request GNOME/gtk!3776

Add more sysprof marks

Add a few more marks during gtk_init to figure out where
our startup time goes, and avoid the sysprof initialization
from distorting the first mark.

Merge branch 'wip/otte/gleanup' into 'master'

reorganize GDK's GL code

See merge request GNOME/gtk!3726

glx: Get rid fo DrawableInfo

Store the frame counter in the surface, where it belongs.

gdkdisplay-win32.c: Don't call ReleaseDC() unnecessarily

The dummy Win32 window that we use to capture display change events and
to create dummy WGL contexts was created with CS_OWNDC, so we really do
not need to (and should not) call ReleaseDC() on the HDC that we
obtained from it, so drop these calls.

GDK-Win32: init_gl(): Default to WGL for now

Since the shaders need to be updated for using with GLES (libANGLE at
least), default to WGL for now.  Unfortunately it is not that common for
Windows to have GLES support, in which the easiest way to obtain such
support is via Google's libANGLE.

GDK-Win32: Fixup, and cleanup WGL HDC handling

It turns out that the problem of the WGL window not drawing was due to
the fact that I messed up where I placed SwapBuffers() during the
conversion... doh:|

At the same time, stop storing the HDC in the GdkWin32GLContextWGL, but
instead always create it along the surface we created, so that it is ready
for use for operating with WGL when we are not dealing with "surfaceless"
contexts.  If we are dealing with "surfaceless" contexts, just use the
HDC of the dummy window that we created when we created the
Gdk(Win32)Display.

WGL contexts should now be in working order at this point.

Split out WGL/EGL stuff and simplify things

This commit attempts to split GdkWin32GLContext into two parts, one for
WGL and the other for EGL (ANGLE), and attempts to simplify things a
bit, by:

*  We are already creating a Win32 window to capture display changes,
   so we can just use that to act as our dummy window that we use to
   find out the pixel format that the system supports for WGL.  We also
   use it to obtain the dummy legacy WGL context that we will always
   require to create our more advanced Core WGL contexts.

*  Like what is done in X11, store up the WGL pixel format or the
   EGLConfig in our GdkWin32Display.

*  Ensure we do not create the dummy WGL context unnecessarily.

In this way, we can successfully create the WGL/EGL contexts, however
there are some issues at this point:

*  For WGL, the code successfully initializes and realizes the WGL
   Contexts, but for some reason things became invisible.  When running
   gtk4-demo, this can be verified by seeing the mouse cursor changing
   when moved to spots where one can resize the window, although they
   were invisible.

*  For EGL, the code initializes EGL but could not realize the EGL
   context as shaders failed to compile.  It seems like the shader issue
   is definitely outside the scope of this MR.

gl: Fix issue with EGL + nvidia

nvidia sets the default draw buffer to GL_NONE if EGL contexts are
initially bound to EGL_NO_SURFACE which is exactly what we are doing. So
bind them to GL_BACK when drawing, as they should be.

See https://phabricator.services.mozilla.com/D118743 for a discussion
about EGL_NO_CONTEXT and draw buffers.

GDK: Add debug option for WGL usage

This way, one can force using WGL on Windows even if EGL support was
enabled.  Also update the help text for gl-egl as it will apply for
Windows, albeit a bit later.

GDK-Win32: Move some functions out of gdkglcontext-win32.c

We want these to be out of the way as we split between WGL and EGL/ANGLE
GL contexts, so that they don't get in the way.

display: Move GL selection code into gdkdisplay-x11.c

This has the benefit that we can refactor it and make sure we deal with
GdkDisplay::init_gl() not being called at all because
GDK_DEBUG=gl-disable had been specified.

x11: Move code from GdkGLContextX11

It's not used there, but both backends have independent
immplementationgs for it.

I want to get rid of GdkGLContextX11 and moving code from it is the
first step.

inspector: Add support for displaying GL errors

Also, no need to show a GL version if GL isn't available.

gdk: Move GL context construction to GdkGLContext

Now that we have the display's context to hook into, we can use it to
construct other GL contexts and don't need a GdkSurface vfunc anymore.

This has the added benefit that backends can have different GdkGLContext
classes on the display and get new GLContexts generated from them, so
we get multiple GL backend support per GDK backend for free.

I originally wanted to make this a vfunc on GdkGLContextClass, but
it turns out all the abckends would just call g_object_new() anyway.

gl: Move vfunc

Instead of
  Display::make_gl_context_current()
we now have
  GLContext::clear_current()
  GLContext::make_current()

This fits better with the backends (we can actually implement
clearCurrent on macOS now) and makes it easier to implement different GL
backends for backends (like EGL/GLX on X11).

We also pass a surfaceless boolean to make_current() so the calling code
can decide if a surface needs to be bound or not, because the backends
were all doing whatever, which was very counterproductive.

gl: Require EGL_KHR_surfaceless_context with egl

The code to create and manage a fake egl surface to bind to is
complex and completely untested because everyone seems to support this
extension.

nvidia and Mesa do support it and according to Mesa devs, adding support
in a new driver is rather simple and Mesa drivers gain that feature
automatically, so all future drivers shoould have it.

gdk: Get rid of paint contexts

... or more exactly: Only use paint contexts with
gdk_cairo_draw_from_gl().

Instead of paint contexts being the only contexts who call swapBuffer(),
any context can be used for this, when it's used with
begin_frame()/end_frame().

This removes 2 features:

1. We no longer need a big sharing hierarchy. All contexts are now
   shared with gdk_display_get_gl_context().
2. There is no longer a difference between attached and non-attached
   contexts. All contexts work the same way.

gl: Store the context::in-frame state in the current context

Do not treat the context as already current when the value
of context::in-frame changes.

This is so we can bind to EGL_NO_SURFACE if context::in-frame == false
and to context::surface if context::in-frame == true.

This allows getting rid of the attached property in future commits.

gl: Deprecate gdk_gl_context_get_shared_context()

It's not used anymore. And in particular we do want to keep the display
context private, so we're not gonna return it from this function.